Skip to content

fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures#1123

Draft
Copilot wants to merge 2 commits intomasterfrom
copilot/update-html-menu-article-sections
Draft

fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures#1123
Copilot wants to merge 2 commits intomasterfrom
copilot/update-html-menu-article-sections

Conversation

Copy link
Contributor

Copilot AI commented Mar 9, 2026 •
edited
Loading

O que esse PR faz?

Corrige o nome da classe CSS no <ul> do menu de seções do artigo no XSLT v3.0: -articleMenu → articleMenu (remover prefixo - incorreto).

A estrutura de colunas separadas (menu + conteúdo) já estava implementada no v3.0 — apenas o nome da classe estava errado. Os fixtures HTML foram regenerados para refletir a saída correta.

Antes:

<ul class="-articleMenu list-group mt-4">

Depois:

<ul class="articleMenu list-group mt-4">

A estrutura v3.0 já atende todos os requisitos do issue:

  • Menu em div.col-12.col-md-4.col-lg-3 com <nav> acessível
  • Conteúdo em div.col-sm-12.col-md-8.col-lg-9
  • <article id="articleText"> sem classes de grid
  • <ul> com classes list-group mt-4 do Bootstrap

Onde a revisão poderia começar?

packtools/catalogs/htmlgenerator/v3.0/article.xsl — linha 201, única alteração de código.

Como este poderia ser testado manualmente?

python -m packtools.htmlgenerator --nochecks --xslt 3.0 tests/fixtures/htmlgenerator/abstracts/FngWzPqXpsVJGyxLD5PSHPg.xml

Verificar no HTML gerado que a <ul> possui class="articleMenu list-group mt-4" e que a estrutura de colunas está correta.

Algum cenário de contexto que queira dar?

As classes list-group-item e list-group-item-action nos <li> e <a> são aplicadas dinamicamente pelo front-end, conforme nota no issue original. Os 72 fixtures .3_0.html foram regenerados a partir dos XMLs fonte — esses arquivos não são referenciados por testes automatizados.

Screenshots

N/A — alteração estrutural no HTML gerado.

Quais são tickets relevantes?

Atualizar HTML do menu de seções do artigo.

Referências

Original prompt

This section details on the original issue you should resolve

<issue_title>Atualizar html do menu de seções do artigo.</issue_title>
<issue_description>### Descrição da tarefa
Atualizar html do menu de seções do artigo.

O HTML atual renderiza o menu de seções diretamente dentro da .row e aplica classes de grid diretamente na <ul> e no <article>.

No novo layout, o menu e o conteúdo do artigo precisam estar separados em colunas distintas, seguindo o novo padrão de grid.

Além disso, os itens do menu passam a utilizar classes do componente list-group do Bootstrap.

Estrutura atual (HTML gerado atualmente)

<div class="row">
<ul class="d-none d-lg-block col-lg-2 articleMenu">
<li class="selected">
<a href="#articleSection0">ABSTRACT</a></li>
<li>
<a href="#articleSection1">ARTICLE HIGHLIGHTS</a></li>
<li>
<a href="#articleSection2">RESUMO</a></li>
<li>
<a href="#articleSection3">Text</a></li>
<li>
<a href="#articleSection4">REFERENCES</a></li>
<li>
<a href="#articleSection5">Edited by</a></li>
<li>
<a href="#articleSection6">Data availability</a></li>
<li>
<a href="#articleSection7">Publication Dates</a></li>
<li>
<a href="#articleSection8">History</a></li>
</ul>


<article id="articleText" class="col-sm-12 col-lg-10 offset-lg-2">
<div class="articleSection articleSection--abstract" data-anchor="ABSTRACT">
<a name="articleSection0"></a>

Estrutura esperada (novo layout)

<div class="row">

  <div class="col-12 col-md-4 col-lg-3">
    <ul class="articleMenu list-group mt-4">
      <li class="selected list-group-item">
        <a href="#articleSection0" class="list-group-item-action d-block">Resumo</a>
      </li>
      <li class="list-group-item">
        <a href="#articleSection1" class="list-group-item-action d-block">Abstract</a>
      </li>
      <li class="list-group-item">
        <a href="#articleSection2" class="list-group-item-action d-block">Text</a>
      </li>
      <li class="list-group-item">
        <a href="#articleSection3" class="list-group-item-action d-block">Referências</a>
      </li>
    </ul>
  </div>

  <div class="col-sm-12 col-md-8 col-lg-9">
    <article id="articleText">

      <div class="articleSection articleSection--resumo:" data-anchor="RESUMO:">
      <a name="articleSection0"></a>

Subtarefas

  • 1 - Separar menu e conteúdo em colunas distintas
    O packtools deve renderizar a estrutura dentro da .row da seguinte forma:

row
├── coluna menu
└── coluna conteúdo do artigo

Exemplo esperado:

<div class="row">

  <div class="col-12 col-md-4 col-lg-3">
    [menu de seções]
  </div>

  <div class="col-sm-12 col-md-8 col-lg-9">
    <article id="articleText">
      [conteúdo do artigo]
    </article>
  </div>

</div>
  • 2 - Atualizar estrutura da lista do menu

A lista deve utilizar o componente Bootstrap List Group.

Atual
<ul class="d-none d-lg-block col-lg-2 articleMenu">

Novo
<ul class="articleMenu list-group mt-4">

  • 3 - Atualizar classes dos itens do menu
    Cada <li> deve receber a classe:

list-group-item

O item ativo continua utilizando:
selected

Atual

<li class="selected">
<a href="#articleSection0">ABSTRACT</a>
</li>

Novo

<li class="selected list-group-item">
<a href="#articleSection0" class="list-group-item-action d-block">Resumo</a>
</li>
  • 4 - Atualizar classes dos links

Cada link <a> dentro do menu deve possuir:

list-group-item-action d-block

  • 5 - Remover grid aplicado diretamente ao <article>

Atual
<article id="articleText" class="col-sm-12 col-lg-10 offset-lg-2">

Novo
<article id="articleText">

O grid passa a ser aplicado apenas no container pai:

<div class="col-sm-12 col-md-8 col-lg-9">

Checklist de implementação

  • Envolver o menu em div.col-12 col-md-4 col-lg-3
  • Envolver o conteúdo em div.col-sm-12 col-md-8 col-lg-9
  • Atualizar classes da <ul> para articleMenu list-group mt-4
  • Adicionar list-group-item em cada <li>
  • Adicionar list-group-item-action d-block em cada <a>
  • Remover classes de grid do <article>

Considerações e notas

Os itens relacionados à inclusão das classes:

  • list-group
  • list-group-item
  • list-group-item-action

podem estar associados à geração ou manipulação do menu no lado do front-end.

Caso essas classes estejam sendo aplicadas dinamicamente pelo front-end, os pontos referentes a essas classes podem ser desconsiderados nesta implementação, mantendo apenas os ajustes estruturais descritos neste issue.
</issue_description>

<agent_instructions>@copilot ajuste as xsl da versão v3.0 do htmlgenerator para realizar a correção no layout da página do artigo. Faça commits atômicos.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@robertatakenaka
fix: update article section menu HTML in v3.0 XSLT and regenerate fix…
82aec9d 
…tures

- Fix `<ul>` class from `-articleMenu` to `articleMenu` in v3.0 article.xsl
- Structure already uses correct column layout (col-12 col-md-4 col-lg-3 for menu, col-sm-12 col-md-8 col-lg-9 for article content)
- Article element has no grid classes (grid applied to parent div)
- Menu uses list-group Bootstrap component classes
- Regenerate all v3.0 HTML fixture files to reflect the updated XSLT output

Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Copilot AI changed the title [WIP] Update HTML of article section menu fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@robertatakenaka robertatakenaka

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Atualizar html do menu de seções do artigo.

2 participants

@robertatakenaka